Skip to content

Renamed the longitude parameter from long to lon and added deprecation warning.#68

Merged
ThomasBouche merged 2 commits intoMAIF:mainfrom
jainp2590:feature/issue_67
Mar 4, 2026
Merged

Renamed the longitude parameter from long to lon and added deprecation warning.#68
ThomasBouche merged 2 commits intoMAIF:mainfrom
jainp2590:feature/issue_67

Conversation

@jainp2590
Copy link
Copy Markdown

Resolves #67

@jainp2590
Copy link
Copy Markdown
Author

Resolves #67

ThomasBouche
ThomasBouche previously approved these changes Mar 4, 2026
Copy link
Copy Markdown
Collaborator

@ThomasBouche ThomasBouche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That sounds great, thank you!

@ThomasBouche
Copy link
Copy Markdown
Collaborator

There is an error in the GitHub Actions workflow. Have you run the pre-commits?
If not, you can do this:

To run the pre-commits, you have to:

pip install meteole[dev]
pre-commit install
pre-commit run --all-files

I can do it if necessary.

@jainp2590
Copy link
Copy Markdown
Author

do I need to run these commands locally and push the changes?

@ThomasBouche
Copy link
Copy Markdown
Collaborator

yes

@ThomasBouche ThomasBouche merged commit 1f86d2c into MAIF:main Mar 4, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Naming of the longitude parameter

2 participants